Skip to content

Conversation

@ryanmac
Copy link
Owner

@ryanmac ryanmac commented Jul 25, 2025

Summary

This PR enhances the Code Conductor installer with automatic pull request creation functionality, addressing the limitation where installations only exist locally until manually committed and pushed.

Problem

When Code Conductor is installed, the .conductor directory and configuration files are created locally. Other agents or developers cannot see or use the system until these changes are merged into the main branch. This creates a manual step that can be overlooked, especially in automated environments like Claude Code.

Solution

Added new command-line flags to conductor-init.sh:

  • --create-pr: Automatically creates a pull request after installation
  • --auto-merge: Enables auto-merge on the created PR (implies --create-pr)
  • --pr-branch <name>: Allows specifying a custom branch name

Changes

Installer Script (conductor-init.sh)

  • Added PR creation logic after the commit step
  • Automatically creates a feature branch if on main/master
  • Pushes changes and creates a descriptive PR
  • Optionally enables auto-merge for seamless activation
  • Adds appropriate labels (conductor:setup, automation)

Documentation Updates

  • README.md: Added PR creation example for Claude Code users
  • docs/INSTALLATION.md:
    • Added new flags to installation options
    • Created dedicated "Pull Request Workflow" section explaining the feature
    • Updated examples to show PR creation usage
  • CLAUDE_CODE_PROMPT.md: Updated all installation commands to use --create-pr --auto-merge

Benefits

  1. Immediate Visibility: Code Conductor becomes visible to all agents/developers as soon as the PR is created
  2. Automation-Friendly: Perfect for Claude Code and other automated environments
  3. Audit Trail: Clear git history showing when Code Conductor was installed
  4. Zero Manual Steps: With auto-merge, the entire installation is hands-free

Testing

The installer checks for:

  • GitHub CLI availability and authentication
  • Whether we're on the default branch (creates new branch if so)
  • Successful push before attempting PR creation
  • Graceful fallback if any step fails

Example Usage

# Install with automatic PR and auto-merge
curl -fsSL https://raw.githubusercontent.com/ryanmac/code-conductor/main/conductor-init.sh | bash -s -- --auto --create-pr --auto-merge

# Install with PR but manual merge
curl -fsSL https://raw.githubusercontent.com/ryanmac/code-conductor/main/conductor-init.sh | bash -s -- --auto --create-pr

# Specify custom branch name
curl -fsSL https://raw.githubusercontent.com/ryanmac/code-conductor/main/conductor-init.sh | bash -s -- --auto --create-pr --pr-branch my-conductor-setup

Related Issues

This addresses the user feedback about needing to merge the .conductor directory before other agents can see it.

🤖 Generated with Claude Code

Co-Authored-By: Claude [email protected]

- Add --create-pr flag to create a pull request after installation
- Add --auto-merge flag to enable auto-merge on the created PR
- Add --pr-branch flag to specify custom branch name
- Update documentation to recommend PR creation for Claude Code
- Explain why PR creation is important for multi-agent visibility

This ensures that Code Conductor installations are immediately visible
to all agents and developers by creating a PR that can be reviewed
and merged, rather than just existing in a local working copy.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
@ryanmac ryanmac merged commit 769da76 into main Jul 25, 2025
10 checks passed
@ryanmac ryanmac deleted the improve-conductor-installer-for-claude branch July 25, 2025 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants